-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Remove useStyles from HURUMap #1008
Remove useStyles from HURUMap #1008
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Headings look good now @kelvinkipruto but Select looks off. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Mostly looks good to me 🚀 I only have a few very minor issues
- There’s an issue with the Pin and Compare component when focused, I can see the shadow/tint slightly overlaps the border radius. Although this is visible in the deployed app, it’s more noticeable in the current version 👇
"& .MuiSelect-filled:focus": {
borderRadius: theme.typography.pxToRem(6),
},
- Some of these icons look like they are cut at the bottom, Not sure why this is happening though might be a good idea to fix them on the same PR, and I was just thinking out loud since we are using 𝕏 instead of the bird as one of the social media icons on the footer, don't we need to update this one as well 🤔
@kilemensi @m453h @koechkevin Thi si ready fo another round of reviews. |
apps/climatemappedafrica/src/components/HURUmap/CategoryHeader/index.js
Outdated
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just (2) minor issues but looks good to me
apps/climatemappedafrica/src/components/HURUmap/ChartFilter/index.js
Outdated
Show resolved
Hide resolved
Signed-off-by: Kipruto <[email protected]>
Description
This PR removes
useStyles
from HURUMapType of change
Screenshots
Checklist: